Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add code for reading zipped logfile #111

Merged
merged 1 commit into from
Apr 3, 2023

Conversation

YoheiKakiuchi
Copy link
Contributor

zipや tarしたlogファイルでプロットできるようにしました。
hrpsys-baseのこのPR fkanehiro/hrpsys-base#1327 で作ったファイルが読めます。

@kindsenior
Copy link
Owner

ありがとうございます

zipやtarをそのまま引数として渡すとディレクトリを展開することなくそのままプロットできるという解釈で合ってますか?

@YoheiKakiuchi
Copy link
Contributor Author

zipやtarをそのまま引数として渡すとディレクトリを展開することなくそのままプロットできるという解釈で合ってますか?

そうです。
fkanehiro/hrpsys-base#1327 hrpsys-sysのDataLoggerの方もzipで出してくれるようにして、
一つのファイルでログが扱えて便利になるかなと。
既存のlogファイルをtarやzipにしても読めるようになっているかと思います。

@kindsenior
Copy link
Owner

そうですね 画期的だと思ってます!
念のため確認ですけど,手動でzipしたファイルでも使えますよね?
その場合,fkanehiro/hrpsys-base#1327よりも先にmergeしようかと思ってます

@YoheiKakiuchi
Copy link
Contributor Author

念のため確認ですけど,手動でzipしたファイルでも使えますよね?

はいできます。

zipファイルの仕様は、
例えば fname.zip という名前で、その zipの中にディレクトリなしで fname.abc_q のような名前でhrpsysのログが入っているというものです。
ここで、fname は log_plotterの -f で与えているのと同じ名前です。

以下のスクリプトをhrpsysのログが入ったディレクトリで実行すると、上記のzipを作ってくれます。
fnameに ドット'.' が含まれると上手く動かない可能性が高いです。
https://gist.github.com/YoheiKakiuchi/a892471485a9f6201d9a4fd739db5bdf

@kindsenior kindsenior merged commit e81b104 into kindsenior:master Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants